Skip to content

Feature feed injector #31

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 22 commits into from
Jun 1, 2025
Merged

Feature feed injector #31

merged 22 commits into from
Jun 1, 2025

Conversation

fulleni
Copy link
Member

@fulleni fulleni commented Jun 1, 2025

Description

Type of Change

  • ✨ New feature (non-breaking change which adds functionality)
  • 🛠️ Bug fix (non-breaking change which fixes an issue)
  • ❌ Breaking change (fix or feature that would cause existing functionality to change)
  • 🧹 Code refactor
  • ✅ Build configuration change
  • 📝 Documentation
  • 🗑️ Chore

fulleni added 22 commits June 1, 2025 02:20
- Inject FeedInjectorService
- Pass AppBloc instance
- Simplify bloc creation
- Inject ads and account actions
- Respect user roles and config
- Handle guest, standard, premium users
- Implement account action variants
- Added Ad and AccountAction support
- Unified item list for search results
- Improved list item spacing
- Changed type to FeedItem
- Renamed results to items
- Inject feed items into search results
- Added AppBloc and FeedInjectorService
- Updated state management for feed items
- Display ads in headlines feed
- Added account actions support
- Improved feed items handling
- Renamed headlines to feedItems
- Updated HeadlinesFeedLoaded state
- Integrated FeedInjectorService
- Added AppBloc dependency
- Updated feed item loading logic
- Handled missing AppConfig
- Added FeedInjectorService
- Display ads and account actions
- Refactored list building
- Displayed generic loading/error
- Renamed headlines to feedItems
- Updated headlinesStatus
- Updated headlinesCursor
- Inject items into feed
- Use FeedInjectorService
- Handle app config
- Update pagination logic
- Added app config repository
- Injected into app bloc
- Added locale property
- Added appConfig property
- Updated copyWith method
- Added _AppConfigFetchRequested event
- Triggers fetching AppConfig
- Fetches AppConfig on user change
- Uses default if not found on backend
- Handles fetch errors gracefully
- Updated link account descriptions
- Updated upgrade account descriptions
- Modified placeholder ad appearance
- Dispatch event after pagination
- Dispatch event after new search
- Added tracking for AccountAction
- Tracks when action is displayed
- Prevents repeated impressions
- Track account action shown
- Dispatch event to app bloc
- Tracks account action shown
- Updates user timestamp
- Added event for user action
- Updates user last shown timestamp
- Backend update pending
@fulleni fulleni merged commit 5ade1f6 into main Jun 1, 2025
0 of 2 checks passed
@fulleni fulleni deleted the feature_feed_injector branch June 1, 2025 01:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant